Skip to content

Remove the assumption that id columns should be the first in read#268

Merged
HTHou merged 1 commit intodevelopfrom
no_assume_id_column_first
Oct 18, 2024
Merged

Remove the assumption that id columns should be the first in read#268
HTHou merged 1 commit intodevelopfrom
no_assume_id_column_first

Conversation

@jt2594838
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 92.85714% with 1 line in your changes missing coverage. Please review.

Project coverage is 63.55%. Comparing base (f8480cc) to head (391b554).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
...a/org/apache/tsfile/file/metadata/TableSchema.java 92.30% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #268      +/-   ##
===========================================
+ Coverage    62.15%   63.55%   +1.39%     
===========================================
  Files          493      495       +2     
  Lines        32439    31745     -694     
  Branches      4232     4110     -122     
===========================================
+ Hits         20162    20175      +13     
+ Misses       11823    11115     -708     
- Partials       454      455       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@HTHou HTHou merged commit 8d94651 into develop Oct 18, 2024
@HTHou HTHou deleted the no_assume_id_column_first branch October 18, 2024 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants